Skip to content

Conversation

@blva
Copy link
Collaborator

@blva blva commented Oct 31, 2025

Proposed changes

  • add local atlas tests to forks

Checklist

@coveralls
Copy link
Collaborator

coveralls commented Oct 31, 2025

Pull Request Test Coverage Report for Build 19275542785

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.04%) to 80.294%

Totals Coverage Status
Change from base Build 19272242315: -0.04%
Covered Lines: 6488
Relevant Lines: 7969

💛 - Coveralls

@blva blva marked this pull request as ready for review October 31, 2025 14:24
@blva blva requested a review from a team as a code owner October 31, 2025 14:24
Copilot AI review requested due to automatic review settings October 31, 2025 14:24
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR enables Atlas Local tests to run on forked repositories, specifically for Dependabot pull requests. The change allows better test coverage for dependency updates from external contributors.

Key changes:

  • Added a new GitHub Actions job to run Atlas Local integration tests on forks when triggered by Dependabot

run: npm ci
- name: Run tests
run: npm test -- tests/integration/tools/atlas-local/
- name: Upload test results
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not an issue with this PR, but we don't really need the coverage uploaded, so we could drop this step from the local tests, as well as the one in the mongodb tests.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thx, done!

@blva blva requested a review from nirinchev November 11, 2025 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants